Skip to content

Conversation

@jglick
Copy link
Member

@jglick jglick commented Jan 3, 2023

#1667 (comment) and in many other cases plugins will be 2.361.x+, so it is cumbersome to update them past that switch (jenkinsci/plugin-pom#478), and older BOM releases will become increasingly quiescent anyway. Will probably also unblock #1647 though I am not attempting that here.

@jglick jglick added the breaking label Jan 3, 2023
@jglick jglick requested review from basil and timja January 3, 2023 22:23
@jglick jglick added the full-test Test all LTS lines in this PR and do not halt upon first error. label Jan 3, 2023
schedule:
interval: "daily"
ignore:
# TODO until 2.332.x & 2.346.x dropped
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was obsolete as of #1642 anyway.

if (art == null) {
if (ga.contains('.plugins') // TODO without an Artifact, we have no reliable way of checking whether it is actually a plugin
&& !(ga == 'io.jenkins.plugins:ionicons-api' && settings.activeProfiles.any {it ==~ /^2[.](332)[.]x$/})) { // TODO: Remove once 2.332.x is no longer part of the BOM (or if MNG-5600 is fixed and we can exclude this dependency in the BOM for old LTS lines)
if (ga.contains('.plugins')) { // TODO without an Artifact, we have no reliable way of checking whether it is actually a plugin
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jglick jglick requested a review from MarkEWaite January 3, 2023 22:26
@jglick jglick enabled auto-merge (squash) January 3, 2023 22:42
Copy link
Contributor

@MarkEWaite MarkEWaite left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks very good to me

@jglick jglick merged commit 0bf2af7 into jenkinsci:master Jan 4, 2023
@jglick jglick deleted the java11 branch January 4, 2023 01:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking full-test Test all LTS lines in this PR and do not halt upon first error.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants